Skip to content

Performance improvement #92

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 28 commits into from

Conversation

fwickert
Copy link

This pull request includes several updates to GitHub workflows, code improvements, and configuration changes. The most important changes involve adding new workflows for advanced CodeQL analysis and .NET CI/CD, updating dependencies, and improving security and documentation.

GitHub Workflows:

  • Advanced CodeQL Analysis: Added a new workflow for advanced CodeQL analysis to scan the codebase for vulnerabilities and errors. (.github/workflows/codeql.yml)
  • .NET CI/CD Pipeline: Introduced a comprehensive CI/CD pipeline for .NET applications, including build, test, Docker build and push, and deployment to different environments (dev, test, prod). (.github/workflows/deploy.yml)
  • Basic .NET Workflow: Added a basic workflow for building and testing .NET projects. (.github/workflows/dotnet.yml)

Code Improvements:

  • Message Analysis Optimization: Simplified the message analysis logic by replacing a loop with a LINQ query to calculate the average word count. (src/Application/src/RazorPagesTestSample/Pages/Index.cshtml.cs)
  • Security Enhancement: Added a check to ensure that files extracted from a ZIP archive do not escape the target directory. (src/Application/src/RazorPagesTestSample/Pages/Index.cshtml.cs)

Configuration and Dependencies:

  • Dockerfile for .NET Application: Added a Dockerfile to build and run the .NET application using .NET 8.0. (src/Application/src/RazorPagesTestSample/dockerfile)
  • Dependency Updates: Updated Newtonsoft.Json to version 13.0.3 in the test project. (src/Application/tests/RazorPagesTestSample.Tests/RazorPagesTestSample.Tests.csproj)

Documentation and Security:

  • Security Policy Update: Revised the SECURITY.md file to provide updated instructions for reporting security issues. (SECURITY.md)
  • Code Ownership: Added a CODEOWNERS file to designate code owners for specific directories. (CODEOWNERS)

fwickert and others added 28 commits October 29, 2024 12:03
Bumps [rexml](https://github.com/ruby/rexml) from 3.3.6 to 3.3.9.
- [Release notes](https://github.com/ruby/rexml/releases)
- [Changelog](https://github.com/ruby/rexml/blob/master/NEWS.md)
- [Commits](ruby/rexml@v3.3.6...v3.3.9)

---
updated-dependencies:
- dependency-name: rexml
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
@fwickert fwickert closed this Oct 30, 2024
@fwickert fwickert deleted the performance-improvement branch October 30, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant